Skip to content

config: husky + lint-staged 설정 #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 20, 2024
Merged

config: husky + lint-staged 설정 #12

merged 5 commits into from
Jul 20, 2024

Conversation

d0422
Copy link
Contributor

@d0422 d0422 commented Jul 20, 2024

  • husky설정
  • lint-staged설정

참고

lint-staged를 돌릴때 warning인 경우는 일단 커밋이되는 것을 확인하여 console.log가 커밋되는 것을 방지하고자 eslint의 no-console을 error로 바꾸었습니다.

@d0422 d0422 requested a review from HBSPS July 20, 2024 01:01
@d0422 d0422 self-assigned this Jul 20, 2024
@d0422 d0422 added the enhancement 성능 개선 label Jul 20, 2024
@d0422 d0422 force-pushed the config/husky-lint-staged branch from d379635 to e6cce73 Compare July 20, 2024 01:11
Copy link
Contributor

@HBSPS HBSPS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@@ -9,6 +9,9 @@
"start": "react-native start",
"test": "jest"
},
"lint-staged": {
"*.{js,ts,tsx}": "eslint --fix"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prettier도 함께 적용하고 캐시처리
하면 좋을 것 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

변경하였습니다!

@d0422 d0422 merged commit a7ba97f into main Jul 20, 2024
1 check passed
@d0422 d0422 deleted the config/husky-lint-staged branch July 20, 2024 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 성능 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants